Hi @Zinggi
Thanks for the PR. Yes, apparently % starts a directive. YamlElixir parses %d.%m.%Y correctly, though...

iex> YamlElixir.read_from_string!("default: %d.%m.%Y")
%{"default" => "%d.%m.%Y"}

Anyway, please fix the test, too. I don't see why we couldn't merge this. WDYT @jlgeering ?

Pull Request Test Coverage Report for Build d5d4f5ef770f205dc6ce138d1efc6df6d19499ee-PR-184

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 100.0%

Totals
Change from base Build 08195b09fe648a028796d2f910c2ed9cff27b690:	0.0%
Covered Lines:	117
Relevant Lines:	117

💛 - Coveralls

Oh, Simplificator is doing Elixir?! Nice!

I don't see why we couldn't merge this

Great, I'll also fix the tests then.

Here is the ruby version to compare with YamlElixir and ymlr. It uses libyaml for it's parser:

irb(main):002> YAML.load("default: %d.%m.%Y")
/home/zinggi/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/psych-5.1.2/lib/psych/parser.rb:62:in `_native_parse': (<unknown>): found character that cannot start any token while scanning for the next token at line 1 column 10 (Psych::SyntaxError)
...

irb(main):003> YAML.load("default: '%d.%m.%Y'")
=> {"default"=>"%d.%m.%Y"}

irb(main):006> YAML.dump({"default"=>"%d.%m.%Y"})
=> "---\ndefault: \"%d.%m.%Y\"\n"

Oh, Simplificator is doing Elixir?! Nice!

Yes, for around 5 years now!

The test almost looks like a copy paste error, it was the only one there without the ' around...

LGTM => I tried to not quote too much "for readability", i.e. there is absolutely no problem in using more quotes when it helps

Thanks a lot!

@Zinggi we published v5.1.2 with your changes!

❤️ Thank you